Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test flow with Ocean Node #49

Merged
merged 40 commits into from
Jun 26, 2024
Merged

Test flow with Ocean Node #49

merged 40 commits into from
Jun 26, 2024

Conversation

jamiehewitt15
Copy link
Member

@jamiehewitt15 jamiehewitt15 commented May 29, 2024 •
edited
Loading

Fixes #43
Fixes: oceanprotocol/ocean-node#473
Fixes: oceanprotocol/ocean-node#323

Changes proposed in this PR:

  • Setting up test flow for publish & download
  • Adding tests for HTTP endpoints
  • Setting up CI
  • Adding linting
  • adding codeowners
  • adding .nvmrc

@jamiehewitt15
Setting up test flow + adding codeowners + adding .nvmrc
087b640
@jamiehewitt15
Updating github actions test flow
eacef33
@jamiehewitt15
Setting up esLint
53ce188
@jamiehewitt15
Fixing eslint errors
2d1bc18
@jamiehewitt15
Fixing dependancy conflict
72c70b0
@jamiehewitt15
Adding workflow for starting barge & ocean node
f9a86f4
@jamiehewitt15
Updating the ci workflow
9500d6a
@jamiehewitt15
Updating github actions flow
4e43326
@jamiehewitt15
pull from origin main
88923b9
@jamiehewitt15
Updating ocean-node setup including the envs
b8758bc
@jamiehewitt15
Adding test for ocean node root endpoint
cf441b8
@jamiehewitt15
Updating CI to run tests
6bfa705
@jamiehewitt15
Updating packages
11067da
@jamiehewitt15
Updating depedanices for tests
d6e9671
@jamiehewitt15
Updating dependancies and getting the tests to run
89fa119
@jamiehewitt15
FIxing linting
76b141b
@jamiehewitt15
Updating test command in the CI
8558308
@jamiehewitt15
Fixing linting
dafe330
@jamiehewitt15
Checking if ocean node is up and running before starting the tests
4f920d1
@jamiehewitt15
Updating CI to checkout develop branch on occean-node git repo
4fbcfd5
@jamiehewitt15
Updating CI
2d0f053
@jamiehewitt15
Updating the CI
f1e626c
@jamiehewitt15
Adding more detailed test of the root endpooint
dfe685e
@jamiehewitt15
Adding test for the direct command status endpoint
cf1f317
@jamiehewitt15
Adding test for getOceanPeers endpoint
9a66e01
@jamiehewitt15
Adding test for Logs endpoint
077bead
@jamiehewitt15
Testing the CLI directly
b36a3b8
@jamiehewitt15
Adding full tests for the setup of ocean-cli
1226e60
@jamiehewitt15
Adding test for publish
4964b19
@jamiehewitt15
Publishing py and js algorithms
e70ccee
@jamiehewitt15 jamiehewitt15 marked this pull request as ready for review June 11, 2024 12:38
@jamiehewitt15 jamiehewitt15 mentioned this pull request Jun 11, 2024
Merged
@paulo-ocean
Copy link
Contributor

seems there are some conflicts to fix
thanks

bogdanfazakas
bogdanfazakas approved these changes Jun 25, 2024
View reviewed changes
Copy link
Member

@bogdanfazakas bogdanfazakas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice job

@jamiehewitt15 jamiehewitt15 merged commit 7c1bb7d into main Jun 26, 2024
3 checks passed
@jamiehewitt15 jamiehewitt15 deleted the node-testing branch June 26, 2024 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bogdanfazakas bogdanfazakas bogdanfazakas approved these changes

@alexcos20 alexcos20 Awaiting requested review from alexcos20 alexcos20 is a code owner

@mariacarmina mariacarmina Awaiting requested review from mariacarmina mariacarmina is a code owner

@paulo-ocean paulo-ocean Awaiting requested review from paulo-ocean paulo-ocean is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

System test: Flows Write tests for HTTP endpoints Add some basic CI flows
3 participants
@jamiehewitt15 @paulo-ocean @bogdanfazakas